-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: remove caching as data is now fetched instantly anyway #36
Conversation
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-ocean-076b69903-36.westeurope.4.azurestaticapps.net |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, aber frag jemandem mit frontend-expertise, wenn du eine echte review willst ;)
}); | ||
const response = await fetch(requestUrl, { | ||
method: "GET", | ||
mode: "cors", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
was macht dieser mode: "cors"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point; ist actually obsolet, weil das der default fetching mode ist, wenn die azure functions angesprochen werden. habs entfernt.
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-ocean-076b69903-36.westeurope.4.azurestaticapps.net |
also, all tables for every +/- 5 years are not loaded at once anymore, significantly reducing the intial loading time. just like the legacy page now :)